Skip to content

Add a shared NuGet icon for the MarkupString packages - #64

Merged
HarryCordewener merged 1 commit into
mainfrom
codex/nuget-logos
Oct 13, 2026
Merged

HarryCordewener merged 1 commit into
mainfrom
codex/nuget-logos

Conversation

@HarryCordewener

Copy link
Copy Markdown
Member

Embed a shared transparent 128×128 NuGet icon in MarkupString, Ansi, Html, Mxp, and Pueblo. The icon uses turquoise angle brackets around three colored text strokes; the existing documentation logo is preserved.

Shared targets select the five published PackageIds and include the icon at the package root. The larger source artwork is retained alongside the package icon.

MarkupString icon

Validation: packed all five packages from current main (v2.18.1), inspected their nuspec icon entries, and verified that each archive contains the exact expected 128×128 PNG. git diff --check passes. No runtime code changed.

@coderabbitai

coderabbitai Bot commented Oct 13, 2026

Copy link
Copy Markdown

Warning

Review limit reached

  • Run on-demand review

This review includes 2 billable files and costs up to $0.50.

  • Ask an admin to make reviews automatic

Open in CodeRabbit

Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing.

Or wait 33 minutes for your next included review.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available. Your 60 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Essentials
  • Run ID: 7253077c-87e1-417c-bdc1-1660d9c4124e
📥 Commits

Reviewing files that changed from the base of the PR and between 0dc73cc and f55a9f6.

⛔ Files ignored due to path filters: 2
  • assets/branding/markupstring-icon.png is excluded by !**/*.png
  • assets/branding/markupstring-logo.png is excluded by !**/*.png
📒 Files selected for processing: 2
  • Directory.Build.props
  • Directory.Build.targets
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@HarryCordewener
HarryCordewener merged commit f06336a into main Oct 13, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant